Skip to content

chore(deps): bump basic-ftp to 5.3.0 and vite to 7.3.2 (replaces #42, #44)#46

Merged
saagpatel merged 1 commit into
masterfrom
codex/chore/deps-second-wave
Apr 21, 2026
Merged

chore(deps): bump basic-ftp to 5.3.0 and vite to 7.3.2 (replaces #42, #44)#46
saagpatel merged 1 commit into
masterfrom
codex/chore/deps-second-wave

Conversation

@saagpatel
Copy link
Copy Markdown
Owner

What

Two dep bumps on top of master. Skipping #43 (picomatch transitive) by design.

  • basic-ftp: pnpm.overrides pin ^5.2.0^5.3.0 (security group)
  • vite: dev dep ^7.0.4^7.3.2 (minor)

Why

These Dependabot PRs were auto-created after the Wave 4 batch (#41 plugin-react) touched the lockfile. Both fail the branch-name governance guard; re-created on a codex-compliant branch per docs/SECURITY.md.

  • Supersedes #42 — npm-security group
  • Supersedes #44 — vite 7.3.1 → 7.3.2
  • Skips #43 — picomatch 2.3.1 → 2.3.2 is transitive-only; pinned by a parent and not worth an override hack for a minor patch. picomatch@4.0.3 already coexists at the latest major.

How

One commit. Edited package.json for the two direct/override entries, then pnpm install to re-resolve. Final diff: 21/21 lines across package.json + pnpm-lock.yaml (only the two target packages' resolution lines).

Testing

  • Commands run: pnpm install, pnpm build, pnpm test, pnpm typecheck
  • Results:
    • pnpm installvite 7.3.1 → 7.3.2, basic-ftp 5.2.0 → 5.3.0 (only these two deps moved)
    • pnpm build → 636ms, dist shape unchanged
    • pnpm test → 127/127
    • pnpm typecheck → clean

Performance impact

  • Bundle delta: none
  • Build time delta: negligible
  • Lighthouse delta: none
  • API latency delta: none
  • DB query delta: none

Risk / Notes

Screenshots (UI only)

  • N/A

Lockfile rationale (if lockfile changed)

  • pnpm-lock.yaml updated only for the two targeted packages; no other versions moved (verified via pnpm's install diff output).

🤖 Generated with Claude Code

Supersedes #42 (npm-security group) and #44 (vite direct bump) —
both blocked by branch-name governance. Re-created on a codex-
compliant branch per docs/SECURITY.md.

- basic-ftp: pnpm.overrides pin ^5.2.0 -> ^5.3.0 (security group)
- vite: devDependency ^7.0.4 -> ^7.3.2 (minor, plugin-react 5 safe)

#43 (picomatch 2.3.1 -> 2.3.2) intentionally skipped: picomatch@2.3.1
is a transitive dep pinned by a parent, and a minor-patch bump isn't
worth an override hack. Picomatch@4.0.3 coexists at the latest major
already.

Validated locally:
- pnpm install resolves cleanly to basic-ftp 5.3.0 and vite 7.3.2
- pnpm build: 636ms
- pnpm test: 127/127
- pnpm typecheck: clean

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@saagpatel saagpatel merged commit 4cd4b9b into master Apr 21, 2026
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants